FIX: Changed formula to calculate zscore#48
Open
CarolynMcNabb wants to merge 1 commit intoCIC-methods:masterfrom
Open
FIX: Changed formula to calculate zscore#48CarolynMcNabb wants to merge 1 commit intoCIC-methods:masterfrom
CarolynMcNabb wants to merge 1 commit intoCIC-methods:masterfrom
Conversation
zmetric=(metric-avg)./stdev in line 76 produced an error when run in Matlab 2015b I updated the code to use zscore: zmetric=zscore(metric) The code now runs with no issues
Contributor
|
Thank you for this! Would you mind pasting in the text of the original error message so that I can see what the issue was? Thanks! |
Contributor
|
The reason I ask is because I think the function 'zscore' is a member of the 'Statistics and Machine Learning Toolbox'. So an implementation involving the 'zscore' function may not work for all users. For that reason, I would prefer to stick with an implementation that does not involve any toolbox functions. So if you show me the error message, maybe I can fix it without using 'zscore'. Thanks! |
Author
|
Hi Jamie,
Sure, no problem at all.
The error message was:
Error using -
Matrix dimensions must agree.
Error in op_rmbadaverages (line 76)
zmetric=(metric-avg)./stdev;
Error in run_megapressproc_auto (line 153)
[out_rm,metric{iter},badAverages]=op_rmbadaverages(out_cc2,nsd,'t');
Thanks for pointing out the use of the toolbox in my suggestion - something
for me to watch out for next time.
Best of luck with the fix!
Carolyn
…On Tue, 20 Apr 2021 at 16:16, Jamie Near ***@***.***> wrote:
The reason I ask is because I think the function 'zscore' is a member of
the 'Statistics and Machine Learning Toolbox'. So an implementation
involving the 'zscore' function may not work for all users. For that
reason, I would prefer to stick with an implementation that does not
involve any toolbox functions.
So if you show me the error message, maybe I can fix it without using
'zscore'.
Thanks!
-Jamie
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#48 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AOAN5WJ35DIYOWV5P5MN3DLTJWLDPANCNFSM43IGHYNQ>
.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
zmetric=(metric-avg)./stdev in line 76 produced an error when run in Matlab 2015b
I updated the code to use zscore:
zmetric=zscore(metric)
The code now runs with no issues